Skip to content

Filter exception musd#8541

Merged
bergarces merged 4 commits intomainfrom
filter-exception-musd
Apr 21, 2026
Merged

Filter exception musd#8541
bergarces merged 4 commits intomainfrom
filter-exception-musd

Conversation

@bergarces
Copy link
Copy Markdown
Contributor

@bergarces bergarces commented Apr 21, 2026

Explanation

MUSD is being filtered out when no occurrences are returned by tokens api.

References

Fixes https://consensyssoftware.atlassian.net/browse/ASSETS-3107

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Low Risk
Very small, targeted change to token filtering logic with added tests; main risk is accidentally allowing similarly formatted asset IDs, but scope is limited to one hardcoded address.

Overview
Prevents the TokenDataSource EVM ERC-20 spam filter (minimum occurrences) from incorrectly hiding the specific mUSD token by adding an explicit allowlist exception for its contract address.

Adds coverage to ensure mUSD is retained even when occurrences is low or missing, and documents the fix in the assets-controller changelog.

Reviewed by Cursor Bugbot for commit 7d0bbd6. Bugbot is set up for automated code reviews on this repo. Configure here.

@bergarces bergarces requested review from a team as code owners April 21, 2026 12:33
@bergarces bergarces added this pull request to the merge queue Apr 21, 2026
Merged via the queue into main with commit 853896f Apr 21, 2026
358 checks passed
@bergarces bergarces deleted the filter-exception-musd branch April 21, 2026 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants